Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Data persistence in candidate views #2613

Merged
merged 3 commits into from
Nov 29, 2024

Conversation

tomlovesgithub
Copy link
Contributor

@tomlovesgithub tomlovesgithub commented Nov 25, 2024

What's included?

Details in the candidate view, (Character Information, Equality and Diversity) were getting stuck when looking through different individuals, in circumstances when there were no answers.
(ie. candidate A has answers for Char Info and E&D survey, Candidate B has none, Navigating to candidate A, then to Candidate B, you see candidate A's answers for Char Info and E&D).

Who should test?

✅ Product owner
✅ Developers
✅ UTG

How to test?

Ensure no answers persist when looking through different candidates.

  • Start from each of the below urls
  • Navigate to the candidate list
  • then to any candidate with 0 applications (or any candidate with no char info/e&d data).
Develop Preview
Candidate with Char Info and E&D Candidate with Char Info and E&D

Risk - how likely is this to impact other areas?

🟢 No risk - this is a self-contained piece of work


PREVIEW:DEVELOP
can be OFF, DEVELOP or STAGING

@tomlovesgithub tomlovesgithub requested review from a team as code owners November 25, 2024 15:15
@tomlovesgithub tomlovesgithub changed the title reset data Data persistence in candidate views Nov 25, 2024
Copy link

github-actions bot commented Nov 25, 2024

Visit the preview URL for this PR (updated for commit dd8d9d9):

https://jac-admin-develop--pr2613-bugfix-candidate-det-uncycwwu.web.app

(expires Sat, 28 Dec 2024 10:37:58 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 4e92cf51659207b0ae3509dc5c40edde50edfec0

Copy link

Visit the preview URL for this PR (updated for commit 5522b6c):

https://jac-apply-admin-production--pr2613-bugfix-candidate-de-9m629uc9.web.app

(expires Wed, 25 Dec 2024 15:19:47 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 2f31b7832d58b04f07d3816f45a5d09d461ded5e

@drieJAC
Copy link
Contributor

drieJAC commented Nov 25, 2024

@tomlovesgithub can you make the testing instructions a bit clearer then I'll test it and code review it? cheers

@tomlovesgithub tomlovesgithub merged commit 90707da into main Nov 29, 2024
6 checks passed
@tomlovesgithub tomlovesgithub deleted the bugfix/candidate-details-persistence branch November 29, 2024 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants